Skip to content

refactor(pack): extract create argument parsing - #1284

Closed
cedric013 wants to merge 1 commit into
yvgude:mainfrom
cedric013:refactor/1277-pack-setup
Closed

refactor(pack): extract create argument parsing#1284
cedric013 wants to merge 1 commit into
yvgude:mainfrom
cedric013:refactor/1277-pack-setup

Conversation

@cedric013

Copy link
Copy Markdown
Contributor

Summary

  • extract pack create argument parsing into focused helpers
  • extract tail-end setup orchestration blocks into named helpers
  • preserve existing permissive CLI parsing behavior

Validation

  • rustfmt --edition 2024 --check src/setup/with_options.rs src/cli/pack_cmd/package.rs
  • git diff --check -- src/setup/with_options.rs src/cli/pack_cmd/package.rs
  • cargo check --bin lean-ctx --quiet

Refs #1277

@yvgude

yvgude commented Jul 27, 2026

Copy link
Copy Markdown
Owner

Review: needs rebase after #1283

PR #1283 (setup phase split) has been merged. This PR overlaps with it on with_options.rs, causing merge conflicts.

Action needed: Please rebase onto main and remove the with_options.rs changes (they're now superseded by #1283). Only the pack-specific changes (pack/create_args.rs, pack/mod.rs) should remain.

Alternatively, I can close this PR and you can re-submit with just the pack refactoring scope.

@yvgude

yvgude commented Jul 28, 2026

Copy link
Copy Markdown
Owner

Integrated via cherry-pick and rebase onto main (commit f433a58d8 → rebased as 6c14ad18d).

Applied the guard-clause refactoring for maybe_add_project_root_warning, maybe_spawn_background_index, and maybe_enable_ide_config_access. Merge conflict with #1283 resolved manually.

Quality gate passed: cargo fmt --check + cargo clippy --all-features -- -D warnings.

Thank you @cedric013 for the contribution!

Refs #1277

@yvgude yvgude closed this Jul 28, 2026
@github-actions github-actions Bot locked and limited conversation to collaborators Jul 28, 2026
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants